Skip to content

Домашнее задание 5.1. Оформление задачи. Разгуляева А.И.#5

Open
ada1ra wants to merge 8 commits intomainfrom
hw_5-1_format
Open

Домашнее задание 5.1. Оформление задачи. Разгуляева А.И.#5
ada1ra wants to merge 8 commits intomainfrom
hw_5-1_format

Conversation

@ada1ra
Copy link
Owner

@ada1ra ada1ra commented Oct 9, 2025

Добавлено домашнее задание 5.1. Оформление задачи.

  • добавлен обновлённый код с задания 1.4. Размен монет
  • добавлен pyproject.toml для ruff

Разгуляева А.И.

@ada1ra ada1ra requested a review from chernishev October 9, 2025 20:39
Copy link

@Godrik0 Godrik0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Кажется, что сейчас в CI у вас нет запуска линтера.
Уделите чуть больше внимания названиям.

Comment on lines 6 to 8
coin1 = 10 # номинал 1-ой монеты
coin2 = 8 # номинал 2-ой монеты
coin3 = 3 # номинал 3-ей монеты
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Константы полностью заглавными буквами в SNAKE_CASE

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[17]: E722 do not use bare 'except'
[20]: E302 expected 2 blank lines, found 1
[46]: E305 expected 2 blank lines after class or function definition, found 1

@ada1ra ada1ra requested a review from Godrik0 December 3, 2025 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants